Skip to content

chore: drop docs/-walk shim in _find_m_standard()#10

Merged
rafael5 merged 1 commit into
mainfrom
chore/drop-m-standard-docs-shim
May 11, 2026
Merged

chore: drop docs/-walk shim in _find_m_standard()#10
rafael5 merged 1 commit into
mainfrom
chore/drop-m-standard-docs-shim

Conversation

@rafael5
Copy link
Copy Markdown
Collaborator

@rafael5 rafael5 commented May 11, 2026

Summary

Follow-up to m-standard PR #6 which relocated docs/integrated/integrated/. The ("", "docs") walk in _find_m_standard() was a compatibility shim for the layout-shift window; with that window closed, it's dead code on any fresh m-standard checkout.

Three-line simplification:

-    for repo in repo_candidates:
-        for sub in ("", "docs"):
-            cand = repo / sub if sub else repo
-            if (cand / "integrated").exists():
-                return cand
+    for repo in repo_candidates:
+        if (repo / "integrated").exists():
+            return repo

Plus docstring trimmed of the layout-shift rationale.

Test plan

  • keyword + doctor + LSP symbol tests green locally (46 passed)
  • CI green on this PR

m-standard merged the docs/integrated/ → integrated/ relocation
(m-standard PR #6), so the `("", "docs")` compatibility walk in
_find_m_standard() is dead code against any fresh m-standard checkout.

Simplification — three lines net:
- one-loop walk over repo candidates
- direct `repo / "integrated"` probe
- docstring trimmed of the layout-shift rationale

Verified with the keyword + doctor + LSP symbol tests (46 passed).
@rafael5 rafael5 merged commit 8bb39bf into main May 11, 2026
1 check passed
@rafael5 rafael5 deleted the chore/drop-m-standard-docs-shim branch May 11, 2026 00:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant